Skip to content

Conversation

@ricardozanini
Copy link
Member

There was an automation error on this PR: #1126

The backport didn't run, and as a result, all PRs related to that issue that came later were also missing in the LTS branch.

I had to verify everything manually. Please do your due diligence and review it thoroughly.

@wmedvede @gmunozfe @gabriel-farache @fjtirado @hbelmiro

gabriel-farache and others added 7 commits June 18, 2025 12:41
…rkiverse#1126)

* Use CredentialProvider in OAUTH2 provider and update data structure for
CrendentialProvider

Signed-off-by: gabriel-farache <[email protected]>

* Fix Custom auth provider doc

Signed-off-by: gabriel-farache <[email protected]>

---------

Signed-off-by: gabriel-farache <[email protected]>
…ception when adding the token to the request header parameters (quarkiverse#1184) (quarkiverse#1189)

* issue-1182: Token propagation throws java.lang.UnsupportedOperationException when adding the token to the request header parameters

* Review comments 1

* Review comments 2

(cherry picked from commit c6e504a)
…kiverse#1192)

* Fix quarkiverse#1191 - refactor CredentialsProvider return type

* Apply feedback

Signed-off-by: gabriel-farache <[email protected]>

* Better log message for oauth2 provider

Signed-off-by: gabriel-farache <[email protected]>

---------

Signed-off-by: gabriel-farache <[email protected]>
@ricardozanini ricardozanini requested a review from a team as a code owner June 18, 2025 16:57
@ricardozanini ricardozanini self-assigned this Jun 18, 2025
@ricardozanini ricardozanini added the area:client This item is related to the client extension label Jun 18, 2025
@ricardozanini
Copy link
Member Author

DO NOT SQUASH MERGE!

@quarkiverse/quarkiverse-openapi-generator please let me handle merging after your approval.

@ricardozanini
Copy link
Member Author

@gabriel-farache, please take a look to see if we are not missing your commits.

Copy link
Contributor

@gabriel-farache gabriel-farache left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like all the commits are there

@fjtirado fjtirado merged commit 5462070 into quarkiverse:main-lts Jun 19, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:client This item is related to the client extension DO NOT MERGE

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants